Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add checkbox #975

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Conversation

tvillegas98
Copy link
Contributor

Motivation

Explain why not how.
Closes #issue_number

Summary of changes

List your changes here.

How to test it?

You can explain how you tested it or suggest a way to do so.

Checklist

  • Tested the changes locally.
  • Reviewed the changes on GitHub, line by line.
  • This change requires new documentation.
    • Documentation has been added/updated.

@tvillegas98 tvillegas98 marked this pull request as ready for review November 4, 2024 18:05
@tvillegas98 tvillegas98 merged commit db35400 into add-shinko Nov 4, 2024
1 check passed
@tvillegas98 tvillegas98 deleted the add-disabled-outside-pool-check branch November 4, 2024 18:06
manucamejo pushed a commit that referenced this pull request Nov 4, 2024
* Apply skill effects to target entities when applying damage to them

* Add shinko to the characters list

* Add shinko skills and params in seeds

* Add underscore to discard unused variable in seeds

* Fix Shinko ulti params and make his pool effects affect enemies even when they escape it

* Make projectiles able to apply an effect on enemies and fix a function taking wrong variable (old effect)

* Fix Shinko basic attack params in seeds

* Add new boolean to effects to differentiate effects inside pool only vs effects still valid outside them

* Fix credo

* Fix credo moving code to new function

* feat: add checkbox (#975)

---------

Co-authored-by: Tomás Villegas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant